Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use daphne (ASGI) as the server #214

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Use daphne (ASGI) as the server #214

wants to merge 2 commits into from

Conversation

zachmullen
Copy link
Contributor

This change will need to coincide with changes to composed-configuration to deal with the problem cited here: https://code.djangoproject.com/ticket/33497

I'll link the relevant related PRs to this one.

@zachmullen
Copy link
Contributor Author

@brianhelba @danlamanna I'd like to get your thoughts on 356b1da. I'm really not sure where to put this configuration logic -- because there's no one-size-fits-all tuning number for the pool size, it can't just be isolated within django-composed-configuration. We could put it there, but have it controlled by an env var, but that would involve also supporting this in our terraform module. Putting it directly in the cookiecutter makes it clear that it could / should be changed on a per-project basis, but the hard-coding feels a little odd.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant